Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Fix] #33 포스트 이미지 보이게 수정 #37

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

JIN921
Copy link
Collaborator

@JIN921 JIN921 commented Nov 13, 2024

1. 무슨 이유로 코드를 변경했나요?

포스트에서 api response를 처리할 때 데이터 구조가 잘못 되어 있어서 이미지 url을 받아오지 못했습니다.
이를 수정하여 이미지를 불러올 수 있도록 고쳤습니다.
image

@JIN921 JIN921 requested a review from woneeeee November 13, 2024 13:10
@JIN921 JIN921 self-assigned this Nov 13, 2024
@JIN921 JIN921 linked an issue Nov 13, 2024 that may be closed by this pull request
};
};

export default function PostImages({ post }: Props) {
if (!post?.Images || post?.Images.length === 0) return null;
if (!post?.images || post?.images.length === 0) return null;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이미지가 없는 경우에 대한 ui 반응은 따로 없는 건가요?

Copy link
Collaborator Author

@JIN921 JIN921 Nov 13, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이미지가 없는 경우는 PostImage 컴포넌트가 안떠서 Post 컴포넌트에 그냥 글만 뜨게 되어 있어요!

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인 했습니다!

};
};

export default function PostImages({ post }: Props) {
if (!post?.Images || post?.Images.length === 0) return null;
if (!post?.images || post?.images.length === 0) return null;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인 했습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[fix] #33 포스트 이미지 보이게 수정
2 participants